[DPE-9205] Update endpoint in pre-refresh check#1378
[DPE-9205] Update endpoint in pre-refresh check#1378
Conversation
Codecov Report❌ Patch coverage is
❌ Your patch check has failed because the patch coverage (0.00%) is below the target coverage (33.00%). You can increase the patch coverage or adjust the target coverage. Additional details and impacted files@@ Coverage Diff @@
## 16/edge #1378 +/- ##
===========================================
- Coverage 70.53% 70.52% -0.02%
===========================================
Files 16 16
Lines 4297 4298 +1
Branches 691 691
===========================================
Hits 3031 3031
- Misses 1055 1056 +1
Partials 211 211 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
|
That actually fixes pre-refresh-check (as always executed on /leader). BTW, does all CI tests passed with the first try? |
For promote-to-primary, we can most probably update the unit peer data. That should trigger an event on the Juju leader and update the endpoints. Should I try as part of this PR? |
Update the relation endpoints when switching over, so that clients don't have to wait for topology observer to update the relation.
Checklist